Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Backport 2.5] added support for QueryStringQuery in rollups #655

Merged
merged 1 commit into from
Jan 10, 2023
Merged

[Backport 2.5] added support for QueryStringQuery in rollups #655

merged 1 commit into from
Jan 10, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 3de566e from #616

@petardz @github-actions
added support for QueryStringQuery in rollups (#616)
7ac0883 
* added support for QueryStringQuery in rollups

Signed-off-by: Petar Dzepina <[email protected]>

* detekt fix

Signed-off-by: Petar Dzepina <[email protected]>

* compile fix

Signed-off-by: Petar Dzepina <[email protected]>

* IT fix

Signed-off-by: Petar Dzepina <[email protected]>

Signed-off-by: Petar Dzepina <[email protected]>
(cherry picked from commit 3de566e)
@opensearch-trigger-bot opensearch-trigger-bot bot requested a review from a team January 10, 2023 01:45
@codecov-commenter
Copy link

codecov-commenter commented Jan 10, 2023 •
edited
Loading

Codecov Report

Merging #655 (3de566e) into 2.5 (25431be) will decrease coverage by 0.14%.
The diff coverage is 67.19%.

❗ Current head 3de566e differs from pull request most recent head 7ac0883. Consider uploading reports for the commit 7ac0883 to get more accurate results

@@             Coverage Diff              @@
##                2.5     #655      +/-   ##
============================================
- Coverage     75.69%   75.55%   -0.15%     
- Complexity     2616     2657      +41     
============================================
  Files           330      333       +3     
  Lines         15115    15395     +280     
  Branches       2338     2384      +46     
============================================
+ Hits          11441    11631     +190     
- Misses         2350     2410      +60     
- Partials       1324     1354      +30
Impacted Files Coverage Δ
...dexmanagement/common/model/notification/Channel.kt 48.38% <0.00%> (ø)
...ndexstatemanagement/migration/MigrationServices.kt 78.26% <ø> (ø)
...ement/model/coordinator/SweptManagedIndexConfig.kt 78.12% <0.00%> (ø)
...nt/indexstatemanagement/model/destination/Chime.kt 77.77% <0.00%> (ø)
...statemanagement/model/destination/CustomWebhook.kt 67.14% <0.00%> (-28.58%) ⬇️
...nt/indexstatemanagement/model/destination/Slack.kt 77.77% <0.00%> (ø)
...t/settings/LegacyOpenDistroManagedIndexSettings.kt 98.56% <ø> (ø)
...action/changepolicy/TransportChangePolicyAction.kt 63.13% <ø> (ø)
...ent/indexstatemanagement/util/ManagedIndexUtils.kt 75.77% <0.00%> (ø)
.../opensearch/indexmanagement/rollup/RollupRunner.kt 59.29% <ø> (ø)
... and 124 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@bowenlan-amzn bowenlan-amzn merged commit 6ed29b5 into 2.5 Jan 10, 2023
@github-actions github-actions bot deleted the backport/backport-616-to-2.5 branch January 10, 2023 05:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bowenlan-amzn bowenlan-amzn bowenlan-amzn approved these changes

@lezzago lezzago lezzago approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@codecov-commenter @lezzago @bowenlan-amzn @petardz